Skip to content

fix: add security insights yaml to known maintainers paths#4388

Merged
mbani01 merged 2 commits into
mainfrom
fix/maintainers_known_paths
Jul 23, 2026
Merged

fix: add security insights yaml to known maintainers paths#4388
mbani01 merged 2 commits into
mainfrom
fix/maintainers_known_paths

Conversation

@mbani01

@mbani01 mbani01 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

This pull request makes a minor update to the MaintainerService class by expanding the set of recognized security insights files. Now, both security-insights.yml and security-insights.yaml are included alongside the previously supported security-insights.md.

  • Added support for security-insights.yml and security-insights.yaml files in the set of recognized security insights files in MaintainerService (maintainer_service.py).

Signed-off-by: Mouad BANI <mouad-mb@outlook.com>
@mbani01 mbani01 self-assigned this Jul 23, 2026
Copilot AI review requested due to automatic review settings July 23, 2026 10:11
@cursor

cursor Bot commented Jul 23, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Adds two static strings to a known-path allowlist; no auth, persistence, or extraction logic changes.

Overview
Extends MaintainerService.KNOWN_PATHS so root-level security-insights.yml and security-insights.yaml are treated like the existing security-insights.md entry.

Repos that publish Security Insights in YAML will get the full-path governance score during candidate ranking, appear in the AI maintainer-file examples, and skip unnecessary section filtering when those paths are selected for extraction.

Reviewed by Cursor Bugbot for commit 876ddcf. Bugbot is set up for automated code reviews on this repo. Configure here.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Jira Issue Key Missing

Your PR title doesn't contain a Jira issue key. Consider adding it for better traceability.

Example:

  • feat: add user authentication (CM-123)
  • feat: add user authentication (IN-123)

Projects:

  • CM: Community Data Platform
  • IN: Insights

Please add a Jira issue key to your PR title.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds YAML Security Insights filenames to maintainer-file detection.

Changes:

  • Recognizes security-insights.yml and security-insights.yaml.

Review note: The PR title requires a trailing JIRA key, e.g. (CM-1234).


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings July 23, 2026 10:22
@mbani01
mbani01 merged commit 69f4fd3 into main Jul 23, 2026
13 checks passed
@mbani01
mbani01 deleted the fix/maintainers_known_paths branch July 23, 2026 10:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants